-
-
Notifications
You must be signed in to change notification settings - Fork 264
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix system tests for when billing is enabled #787
Fix system tests for when billing is enabled #787
Conversation
Because the pricing page can take a while to finish and then redirect, I had to add some extra If anything I can double check on the numbers and try to cut them down if necessary. |
Actually, now that I'm thinking about it we might have to remove |
Ok, I made the sleep times shorter, and since we don't go to the pricing page when a freemium is enabled, I removed the unnecessary checks from the application system case because we skip this inside each system test anyways. |
@gazayas This looks great! Can you resolve the conflicts? |
@jagthedrummer Done! |
Closes #782.
Since our billing page has changed, we needed to update the test helper that fills out the necessary fields to subscribe to a plan.
Old
New
There are still a bunch of system tests that need to be updated, so I'll mark the PR as ready for review once those are done.